We don't need the IS_WIN check yet - for now this part of the script is windows only (we will make it work for linux with the next bug). So please remove it for now. Also, the default values for python_path and nsis_path can be defined as a default= argument in the add_argument call, so you can just remove all this block.

Hm, I see. But when we call bundle under linux, the default argument will be also given to bundle if we forget to give a python path. So, why don't we set different default value of path when we call different bundle funtion?

Well, we will! But it is for the next patch - for now, the bundle command is only used in windows, so let keep things simple.

I mean you can treat this patch as a windows bundle function.

python_path = 'C:\Python27'
nsis_path = 'C:\NSIS'

And, we when bundle be called under Linux system.

python_path = /usr/lib/python2.7

The goal of this patch is to remove the environment variables used to use command line arguments instead. Let's just do this for now, and not something else - it is easy to do, and easy to review. Let's talk on irc about that!

I see. :)

Sorry, I have some classes this afternoon. I will send you email after that. Let's fix this issue first. :D